Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Add a context for testing flatpak #818

Closed
wants to merge 5 commits into from

Conversation

cgwalters
Copy link
Member

This was part of the philosophy behind https://wiki.gnome.org/Initiatives/GnomeGoals/InstalledTests -
libraries like libostree don't need to replicate everything in unit tests, we
can use the tests from our dependencies directly too.

We'll also get API break coverage testing too.

@cgwalters
Copy link
Member Author

Although, this is less useful than it could be because
SKIP: tests/test-run.sh - Cannot run bwrap
which is projectatomic/papr#20

@jlebon
Copy link
Member

jlebon commented Apr 28, 2017

Can we just provision a VM for it for now?

@cgwalters
Copy link
Member Author

We could; that's what the flatpak CI is doing now. I guess the thing that bothers me about that is that it's regressing to the old mindset of build-in-VM instead of build-in-container (and possibly run-in-VM after).

@cgwalters
Copy link
Member Author

That said, maybe it wouldn't be too hard to try to make a reusable script implementing that pattern, like I did for bubblewrap. Then we could copy that around for a while, and it'd give us some more real-world experience for making it declarative in rhci.

@cgwalters
Copy link
Member Author

Oh right, the other problem with that is flatpak's tests are uninstalled. So I'd have to port them.

@jlebon
Copy link
Member

jlebon commented Apr 28, 2017

Hmm, how about we provision a VM and run it all in a privileged container there?

That said, maybe it wouldn't be too hard to try to make a reusable script implementing that pattern, like I did for bubblewrap.

I think that pattern will be obsoleted by adding RPM support to rhci, right? Then you have a bona fide tree with your bits already installed.

This was part of the philosophy behind https://wiki.gnome.org/Initiatives/GnomeGoals/InstalledTests -
libraries like libostree don't need to replicate everything in unit tests, we
can use the tests from our dependencies directly too.

We'll also get API break coverage testing too.
@cgwalters
Copy link
Member Author

Ok, done ⬆️

@cgwalters
Copy link
Member Author

Oh heh right, not quite:

SKIP: tests/test-run.sh - this test requires xattr support

Yep, the only sane thing to do is to change the flatpak tests to be installed. I'd like to do that for ostree too.

distro: fedora/25/atomic

tests:
- docker run --rm --privileged -v $(pwd):/srv/code registry.fedoraproject.org/fedora:25 /bin/sh -c "cd /srv/code && ./ci/flatpak.sh"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we also upload the test results?

artifacts:
  - test-suite.log

And in the flatpak.sh script, have something like:

trap '[ ! -f test-suite.log ] || cp test-suite.log /srv/code' ERR

?

@jlebon
Copy link
Member

jlebon commented Apr 28, 2017

That's weird. The default docker backend is still devmapper for F25, right?

Actually, looking at the output, I don't see attr getting installed at all. So could it just be setfattr missing? I suggested uploading the logs as well to help debug that.

@jlebon
Copy link
Member

jlebon commented Apr 28, 2017

Nice! The error is just flaky infra, so let's give it a try.
@rh-atomic-bot r+ 9ee612b

@rh-atomic-bot
Copy link

⌛ Testing commit 9ee612b with merge e60b9bc...

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: jlebon
Pushing e60b9bc to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants